build: speedup compile times via dev-profile debuginfo, git2 0.20, rustls - #4
Open
jakob1379 wants to merge 2 commits into
Open
build: speedup compile times via dev-profile debuginfo, git2 0.20, rustls#4jakob1379 wants to merge 2 commits into
jakob1379 wants to merge 2 commits into
Conversation
- dev profile: line-tables-only for workspace code, no debuginfo for dependencies and build scripts - git2 0.19 -> 0.20 as a workspace dependency with default-features off: only local git operations are used, so the ssh/https transports, libssh2-sys, and vendored openssl leave the build graph - diffcore-core reqwest: rustls-tls-native-roots (system trust store kept) instead of native-tls; tauri dev-dep reqwest likewise, so cargo test no longer needs system OpenSSL headers Full clean dev build 177.6s -> ~125-135s with plain cargo; 61.1s +/- 0.7s (hyperfine, n=3) with the mold+sccache devShell. 2191 tests pass.
New headless axum binary serves the built UI and mirrors the tauri IPC
surface over HTTP so diffs can be reviewed on a remote server:
- commands decoupled from tauri: cfg-swapped State shim, process-global
background runtime replacing tauri::async_runtime, cfg-gated watchers;
features 'desktop' (default) and 'web' are mutually exclusive
- POST /api/invoke/{cmd} (camelCase args, 1:1 with tauri IPC),
GET /api/health, same-origin SSE job streams, SPA static serving
- Host/Origin allowlist guard (DNS-rebinding/CSRF) + strict JSON bodies;
editor and file-watch commands are desktop-only (501 on web)
- frontend transport priority: Tauri IPC -> web API (health probe) ->
demo mocks; desktop-only UI hidden in web mode
- 8 integration tests for the HTTP surface
Deliberate v1 cuts (see specs/web-server-mode.md): no auth (loopback
bind, tunnel/reverse-proxy for remote), single shared AppState.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Three independent, behavior-preserving changes to the dependency graph and dev profile:
Dev-profile debuginfo cuts (workspace
Cargo.toml): workspace crates emitline-tables-only(backtraces keep file:line), dependencies and build scripts emitnone. Nobody steps through tokio/tauri internals in a debugger; full
debug = 2for750 dep crates was pure cost.
target/shrinks 3.5G → 2.2G.git2 0.19 → 0.20,
default-features = false, moved to[workspace.dependencies]:diffcore only performs local git operations (
Repository::open/discover/init, diffs,statuses, branches — verified by grep, no clone/fetch/push/remote/credential usage
anywhere). Dropping the default
ssh/httpstransport features removeslibssh2-sys,openssl-sys, and the vendored-OpenSSL C build from the graph entirely, and shrinksthe bundled libgit2 build.
reqwest: native-tls →
rustls-tls-native-roots(diffcore-core, and thediffcore-tauri dev-dependency): unifies on the rustls stack tauri-plugin-updater
already compiles, instead of building two TLS stacks.
native-roots(not plainrustls-tls) keeps the **syate proxies / private CAskeep working;
macos-system-configurationkeeps macOS proxy detection. Side benefit:cargo testno longer needsNumbers
Measured with hyperfine on a 16` between runs):
cargo clean -p× 3 crates)Single-shot timings on this workload swing ±25% with thermal state — the hyperfine
numbers are the trustworthy one
Verification
cargo tree -i native-tls/ build graph, dev-depsincluded (native-tls only appears under the optional
embeddingsfeature via hf-hub)requirements); all findings resolved or refuted with measurements
Notes
explicitly preserved via `nat
machines without it); a nix d as a follow-up.